filechooserbutton: When the combo box changes, set the *file*, not the current folder

The currently-selected file *is* the selection even in SELECT_FOLDER mode.  Do not confuse this
with the current folder.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Punjabi: Translation updated (aalam)

Updated Serbian translation

[l10n] Updated German translation

[l10n] Updated German translation

Updated Indonesian translation of po-properties

Updated Indonesian translation

Fix typo reqired -> required

Updated Brazilian Portuguese translation

Updated Brazilian Portuguese translation

Conflicts:
po-properties/pt_BR.po

Updated Brazilian Portuguese translation

treeview: fix a critical warning

gtk_tree_view_column_unset_tree_view() resets column->priv->tree_view to
NULL.
The function is called when a column is removed, but later from the same
function we would call _gtk_tree_view_column_unrealize_button(), which
expects column->priv->tree_view to be != NULL, causing these critical
warnings

Gtk-CRITICAL **: gtk_widget_unregister_window: assertion
`GTK_IS_WIDGET (widget)' failed

This commit moves the call to unset the tree view after the button is
unrealized.

https://bugzilla.gnome.org/show_bug.cgi?id=695473

docs: change id for Wayland section to not conflict with OS X

https://bugzilla.gnome.org/show_bug.cgi?id=695391

Avoid a warning under Wayland

Just having an ifdef is not enough, we need to check the type
of the window at runtime.

Updated Polish translation

Updated POTFILES.in

filechooserbutton: Update the combo box even after the dialog is cancelled

We assumed that we didn't have to update the combo box if the dialog got cancelled,
as it should simply retain its previous contents.  But this assumption doesn't work
as the dialog is brought up with the 'Other...' item - we don't want the
combo box to keep showing 'Other...' if the dialog is cancelled.

The test from the previous commit now passes.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

filechooserbutton: New test for opening the Other item in the combo box and then canceling the dialog

There is this bug:

1. Start with a file chooser button in SELECT_FOLDER mode, and select a folder from the combo box.

2. Click on the button's combo box, select 'Other...'

3. You get the file chooser dialog. Cancel the dialog.

4. The file chooser button's combo box still shows 'Other...' instead of
   showing the selection from (1).

This is a test to ensure that the original selection is restored.
The test fails right now.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Fix the build

This was broken in commit 5bbbc47a4c306653e8347f7afb85a940a503f755

spinbutton: don't override initial text in non-numeric-only spin buttons

https://bugzilla.gnome.org/show_bug.cgi?id=695312

filechooserbutton: Don't show the (None) item when the combo box is popped up

This is surprisingly tricky, since the (None) item *has* to be a visible item while
the combo box is *not* popped up, so that it can show its contents.  But the item
has to be *not* visible when the combo box is popped up.

Also, update the whole button's selection, not just the underlying dialog's, when
the combo box changes its selection - based on a patch by Paul Davis in
https://bugzilla.gnome.org/show_bug.cgi?id=691040#c20

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

filechooserbutton: In SELECT_FOLDER, when the selection is empty, show (None) in the combo box

We do this by adding a new row type, ROW_TYPE_EMPTY_SELECTION.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

filechooserbutton: Update the button/combo at construction time

This way the internal labels will show the correct selection even if nothing
has been selected programmatically.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

filechooserbutton: Update the button/combo whenever the selection is changed programmatically

We didn't change it when the file chooser button's dialog was inactive, and so
the actual file chooser button would not visually reflect the current selection.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

filechooserbutton: Test that the expected filename is shown

We do some gymnastics to pull the string out of the GtkButton or the GtkComboBox that is
being used in GtkFileChooserButton to show the current selection when the dialog
is inactive - namely, we look for the subwidget with the correct ATK role, and pull its
accessible name.

Currently the test fails; this is https://bugzilla.gnome.org/show_bug.cgi?id=691040#c18

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Don't mention gtk_set_locale() in docs; this function does not exist

Finnish translation update by Jiri Grönroos

Updated Brazilian Portuguese translation

Updated Kazakh translation.

Updated Belarusian translation.

Updated Brazilian Portuguese translation

Update Arabic translation

printing: Fix build with -Werror=format-security

GtkTextView: don't popdown a bubble if we don't have one

This causes a segfault.

https://bugzilla.gnome.org/show_bug.cgi?id=695304

GtkEntry: Sanity check the end_pos value in _get_display_text()

GtkLabelAccessible: Sanity check _get_text() input values

gdk/win32/gdkevents-win32.c: Fix build

The variable "display" is not defined, causing the build to fail.
Judging from the code, it seems that it should be _gdk_display instead.

https://bugzilla.gnome.org/show_bug.cgi?id=694339

quartz: Null check title before setting it

This avoids a crash in objective-c.

https://bugzilla.gnome.org/show_bug.cgi?id=695278

Update Visual Studio Solution Files

Integrate the utility projects to build the introspection files into the
main solution files, so that one can build the introspection files from the
IDE.  This is not built by default, so one can build the introspection
files if he/she chooses to do so.

Visual Studio Builds: Support Building Introspection Files

Add Windows .bat and Python script to call g-ir-scanner to build
introspection files for Visual Studio builds.  This will read from the
autotools files using Python REGEX functionality to determine the headers
and sources for g-ir-scanner to process, so the autotools files will not
need to be updated except to distribute the necessary files.  Thils will
also enable one to build introspection files on Windows without using a
BASH-style shell such as MSYS.

Also add an utility Visual Studio project to call the Windows .bat to
build the introspection files for GTK+/GDK, for convenience.

po: Remove fuzzy tag for confirmed translations

filechooser: Use _gtk_file_has_native_path() throughout when testing for local_only

In the case of checking for local_only, g_file_is_native() is not useful, since it
will return FALSE for something in a FUSE mount.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

filechooser: Also convert get_uris() to returning native paths

It was only being done in gtk_file_chooser_get_uri().

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

filechooser: Rename _gtk_file_is_path_not_local() to _gtk_file_has_native_path()

Negatives in names of boolean functions are confusing.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

filechooser: Show FUSE mounted locations in shortcuts

Since FUSE locations can be handled safely by applications show these mounted locations regardless of whether gtk_file_chooser_set_local_only()
is set to TRUE

https://bugzilla.gnome.org/show_bug.cgi?id=586367

Post-release bump

3.7.12

GtkBubbleWindow: use OSD style class

GtkBubbleWindow: rework how drawing is done

Use gtk_render_frame_gap(), refactoring the code fetching coordinates to
be usable by it. This allows for rounded corners in the window shape.

GtkBubbleWindow: allocate CSS borders and paddings

Rename property to be more neutral

Don't put toolbar in the name of the property - we may use
a different container down the road, and then this name
would be misleading.

Really keep GtkBubbleWindow private

_-prefix the symbols to keep them from leaking out.
Also, un-doc-commentize the sources.

Small documentation tweak

Popdown the bubble window when text view is scrolled

Allow to populate bubbles with extra content

We do this by making the ::populate-popup signals a little more
flexible. They used to just accept a GtkMenu as argument, now
they can take a menu or a toolbar. To not break the expectations
of existing callbacks, we only emit ::populate-popup with a toolbar
if the :populate-toolbar property is TRUE.

Drop GtkSelectionWindow

It is no longer used.

Do without GtkSelectionWindow

Just populate a toolbar directly, in GtkEntry and GtkTextView.

Fix a typo

Made GtkBubble/SelectionWindow private at the moment

Use gdk_threads_add_timeout to popup the selection window

Second granularity may bite us back there

Fix includes

gtkbubblewindow.h was including gtk.h, which in turn included
gtkselectionwindow.h, leading to build failure.

Don't allow individual #includes for GtkBubble/SelectionWindow

GtkBubbleWindow: Use style border color to stroke the bubble shape

This improves themeability a bit, corners are still square though...

textview: Use GtkSelectionWindow for touch text selection

This enables touch devices to manipulate the text selection

entry: Use GtkSelectionWindow for touch text selection

This enables touch devices to manipulate the text selection

Add GtkSelectionWindow

This is a helper object to provide context-dependent content
edition apt for touch devices.

Add GtkBubbleWindow

This popup window widget can be used for touch friendly context
menus that point to a concrete area.

texthandle: Set a bigger input shape, covering the line height

Now, even if the handles being rendered are small, the handle touch
input shape will be as wide as the visible part of the rendered asset, and
high enough to cover both the handle and the height of the line where
the selection bound is.

Also, make handles have the same virtual distance to the line top/bottom
when a drag starts, so the handle doesn't jump to another line after a
too short threshold.

Ensure the insertion handle stays on fake events (eg from IM)

Don't set handles mode to none if the event has send_event set.
For consistency with GtkEntry, also make GtkTextView keep the
handle mode on buffer changes.

texthandles: Keep state internally to avoid X overhead

Handles now do sync X calls less often. As visibility state
is kept, it now can move+resize+show handles at once instead
of in separated steps.

Updated Czech translation

Updated Czech translation

entrycompletion: Don't reconnect signals all the time

We block signal handlers areound GtkEntry signal emission and if those
signals get used to call functions on the completion that cause a
reconnection of the signals, then the reconnected signals will not be
blocked anymore (so they might get emitted?) and unblocking the old
signal id will later cause warnings.

Fixes spurious warnings in gtk/tests/filechooser tests.

a11y: Emit active-descendants-changed when focus changes

This fixes the a11y/tree-relationships testcase.

The emission was lost in refactoring in commit
76670734f4330acc28c3c1d885ef944e85beb36e

Updated Thai translation

Updated Brazilian Portuguese translation

Updated Brazilian Portuguese translation proofread by Enrico Nicoletto

x11: Remove unused macro

https://bugzilla.gnome.org/show_bug.cgi?id=694395

bin: initialize out variables to zero for get_preferred_ functions.

Fixes case of child not visible then minimum_width/heigth natural_width
/heigth not initialized. Thus caller gets a random value.

https://bugzilla.gnome.org/show_bug.cgi?id=695131

clipboard: remove dead code

When setting contents of the clipboard and ownership or user data changes, we
end up calling clipboard_unset() to fully cleanup the previous clipboard state.
This call will itself call clear_func() for the previous user_data, and always
reset both 'get_func' and 'clear_func' to NULL.

So it's actually not possible to have 'get_func' being non-NULL once we have
called clipboard_unset(), so just remove that condition check and the code
inside.

https://bugzilla.gnome.org/show_bug.cgi?id=694924

GtkColorChooserWidget: deselect swatch before removing palettes

When adding custom palettes, set the current swatch to NULL if the
palette it is on is going to be removed.

https://bugzilla.gnome.org/show_bug.cgi?id=693995

gtkenums: correct various documentation typos

docs: correct various spelling and grammar errors

I noticed a few cases of "wether", and while fixing them noticed a few
"its" which should be "it's". It all went downhill from there.

GtkIconTheme: fix regression from 0db32f0632ef4675bfcfc9ec201f7af157a48ab0

icon_info_dup() is now called also for GtkIconInfos that already have
a pixbuf, so we must make sure that we correctly carry that from
the original icon_info to the copy.

Updated Polish translation

appchooserdialog: remove redundant checks

This is checked by GIO for us now.
Also, it's generally just a bad idea spawning error dialogs from inside
a library on top of other dialogs: lesson learned.

https://bugzilla.gnome.org/show_bug.cgi?id=675333

GtkIconTheme: fix failed assertion when asynchrnously loading emblemed icons

If you tried to lookup an icon that was not emblemed, and then looked up
an emblemed icon with the same base, we would override the iconinfo adding
the emblems inline. Later, when the icon finished rendering, inside
gtk_icon_info_load_icon_finish, we would copy the result from the duplicate
(which did not include the emblem infos), but the icon would still fail the
assertion, because emblems infos are present but emblem_applied is false
(they were not requested in the first place!).
Solve this by avoiding the overwrite on a cached iconinfo, and instead duplicate
the iconinfo before adding the emblems. It is expected that another layer
of caching (such as StTextureCache in gnome-shell) will take care of avoiding
multiple rendering of the same icon+emblem combination.

https://bugzilla.gnome.org/show_bug.cgi?id=694968

printing: Mark texts for translation correctly

Use _() instead of N_() for printer states. Mark connection
string "; " for translation so it is correctly handled for
right to left languages.

printing: Remove unnecessary break

Remove the first space when connecting multiple printer states
together.

Related to
https://bugzilla.gnome.org/show_bug.cgi?id=688956

printing: List Avahi printers

Show printers advertised by avahi on local network. CUPS
backend now looks for _ipps._tcp and _ipp._tcp services
offered by avahi. If it finds such a service (printer)
it requests its attributes through IPP_GET_PRINTER_ATTRIBUTES
ipp request and adds it to the list of printers. Such printer
behaves like a remote printer then.
If an avahi printer is a default printer then it is considered
default by the backend only if there is no local or remote
default printer.
This functionality is enabled when building Gtk+ with CUPS 1.6
or later because it replaces browsing protocol removed in CUPS 1.6.

https://bugzilla.gnome.org/show_bug.cgi?id=688956

Updated Brazilian Portuguese translation

Place the search icon in the primary slot of the entry

Instead of using the secondary slot for both clear and search. This
Makes it possible to use the search icon for actions regardless of
whether text has been entered, makes it possible to use the primary
icon to indicate search status, allows us to indicate the purpose
of the entry even if text has been already entered.

https://bugzilla.gnome.org/show_bug.cgi?id=694990

Updated Lithuanian properties translation

Updated Vietnamese translation

po/vi: import from Damned Lies

Fix make check

Revert "Fix gtkprintbackendcups crash"

This reverts commit 54ffc982e96bc4b2484a2611c7dc5cf24c4c4a1a.

I've undone the GList behaviour change instead.

treeview: center expander allocated space

This patch centers the expander in the extra space allocated by the
horizontal-separator style property.

https://bugzilla.gnome.org/show_bug.cgi?id=650424

treeview: properly calculate the treeview expander size

It should be expander-size + horizontal-separator / 2. Rework code
calculating the render position of the arrow to account for the larger
size.

https://bugzilla.gnome.org/show_bug.cgi?id=650424

treeview: remove extra padding handling

We'll replace this with a proper calculation.

https://bugzilla.gnome.org/show_bug.cgi?id=650424